Skip to content

chore: updated to Theia 1.35.0 #1976

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 4 commits into from
Closed

Conversation

kittaakos
Copy link
Contributor

@kittaakos kittaakos commented Mar 22, 2023

Motivation

Change description

Closes #1501

readonly_1501.mp4

Other information

Reviewer checklist

  • PR addresses a single concern.
  • The PR has no duplicates (please search among the Pull Requests before creating one)
  • PR title and description are properly filled.
  • Docs have been added / updated (for bug fixes / features)

@kittaakos kittaakos added topic: code Related to content of the project itself topic: theia Related to the Theia IDE framework labels Mar 22, 2023
@kittaakos kittaakos self-assigned this Mar 22, 2023
Copy link
Contributor

@per1234 per1234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know Akos is already aware, but I'll add a note here for transparency:

There was a hypothesis this update might have allowed us to avoid the GLibC dependency that is causing builds of the IDE on a machine with a newer Ubuntu version (e.g., the ubuntu-latest GitHub Actions runner has Ubuntu 22.04) to fail to start on machines with an older version of GLibC, as discussed at #1983 and #259.

I tried starting the build of this PR (created on the ubuntu-latest runner) on a Ubuntu 18.04 machine and encountered the same problem as discussed in #1983:

Error: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_arduinlmTF8J/resources/app/node_modules/nsfw/build/Release/nsfw.node)
    at process.func [as dlopen] (node:electron/js2c/asar_bundle:5:1800)
    at Object.Module._extensions..node (node:internal/modules/cjs/loader:1170:18)
    at Object.func [as .node] (node:electron/js2c/asar_bundle:5:1800)
    at Module.load (node:internal/modules/cjs/loader:982:32)
    at Module._load (node:internal/modules/cjs/loader:823:12)
    at Function.c._load (node:electron/js2c/asar_bundle:5:13331)
    at Module.require (node:internal/modules/cjs/loader:1006:19)
    at require (node:internal/modules/cjs/helpers:93:18)
    at Object.<anonymous> (/tmp/.mount_arduinlmTF8J/resources/app/node_modules/nsfw/js/src/index.js:4:14)
    at Module._compile (node:internal/modules/cjs/loader:1110:14) {
  code: 'ERR_DLOPEN_FAILED'
}

@kittaakos
Copy link
Contributor Author

Replaced by #2027. Closing as invalid.

@kittaakos kittaakos closed this Apr 20, 2023
@kittaakos kittaakos added the conclusion: invalid Issue/PR not valid label Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conclusion: invalid Issue/PR not valid topic: code Related to content of the project itself topic: theia Related to the Theia IDE framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Saving sketch fails silently when files are read-only
2 participants